Update thread sanitizer CI to use a Python 3.15 image#243
Conversation
kumaraditya303
left a comment
kumaraditya303
left a comment
There was a problem hiding this comment.
I can't reproduce the TSan failures under 3.15. There have been a number of thread safety fixes to the standard library since 3.14, so in addition to fixing the issue we're hitting in CI, we can also completely remove the suppressions file as well.
Yes, in main branch all of them are fixed.
This LGTM
|
I am a little uncomfortable using an alpha-release python as the ground truth for the TSAN builds. What does numpy do? |
NumPy uses Python 3.14, although I personally think it'd probably be fine to update those builds.
Fair enough, let me see if I can add a suppression for the issue CI is currently hitting. |
See #244. |
3 participants
I can't reproduce the TSan failures under 3.15. There have been a number of thread safety fixes to the standard library since 3.14, so in addition to fixing the issue we're hitting in CI, we can also completely remove the suppressions file as well.
Also adds test timeouts per the advice here.
Ping @kumaraditya303 since he originally set all this up last year.
Fixes #242.